Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add more info to console error detection in integration tests #881

Merged
merged 4 commits into from
Sep 1, 2022

Conversation

Janpot
Copy link
Member

@Janpot Janpot commented Sep 1, 2022

Make error more visible for debugging purposes

@render
Copy link

render bot commented Sep 1, 2022

@oliviertassinari oliviertassinari requested a deployment to integration-upd - toolpad-db PR #881 September 1, 2022 14:38 — with Render Abandoned
@apedroferreira
Copy link
Member

apedroferreira commented Sep 1, 2022

this was the run that had failed https://app.circleci.com/pipelines/github/mui/mui-toolpad/2858/workflows/1440fe9e-1988-4294-8558-e389259fae6a/jobs/10285

i guess it was something flaky with the test showing a console error but yeah, the message isn't very helpful :D

Copy link
Member

@apedroferreira apedroferreira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@@ -18,14 +19,21 @@ const IGNORED_ERRORS = [

export const test = base.extend({
page: async ({ page }, use) => {
const entries: ConsoleEntry[] = [];
const entriePromises: Promise<ConsoleEntry>[] = [];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small typo :)

@Janpot Janpot enabled auto-merge (squash) September 1, 2022 15:07
@Janpot Janpot merged commit 39bd2c7 into master Sep 1, 2022
@Janpot Janpot deleted the integration-upd branch September 1, 2022 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants